Skip to content

Conversation

@killerwolf
Copy link
Owner

This commit introduces a testing framework to the project using Vitest and JSDOM.

Key changes include:

  • Added vitest and jsdom as development dependencies.
  • Created a vitest.config.js for basic JSDOM environment configuration.
  • Updated npm scripts in package.json to include test and test:watch commands.
  • Implemented initial unit tests for core VisualImageTool functionalities in src/visual-image-tool.test.js. This covers instantiation, focus point retrieval and manipulation, and the destroy method.
  • Updated the GitHub Actions workflow (.github/workflows/code-quality.yml) to include a new job that runs npm test on pushes and pull requests to the main branch.
  • Added a "Running Tests" section to README.md to guide contributors on how to execute the tests.

This addresses issue #8, enhancing the project's reliability and maintainability by providing a solid foundation for testing.

This commit introduces a testing framework to the project using Vitest and JSDOM.

Key changes include:
- Added `vitest` and `jsdom` as development dependencies.
- Created a `vitest.config.js` for basic JSDOM environment configuration.
- Updated npm scripts in `package.json` to include `test` and `test:watch` commands.
- Implemented initial unit tests for core `VisualImageTool` functionalities in `src/visual-image-tool.test.js`. This covers instantiation, focus point retrieval and manipulation, and the destroy method.
- Updated the GitHub Actions workflow (`.github/workflows/code-quality.yml`) to include a new job that runs `npm test` on pushes and pull requests to the main branch.
- Added a "Running Tests" section to `README.md` to guide contributors on how to execute the tests.

This addresses issue #8, enhancing the project's reliability and maintainability by providing a solid foundation for testing.
This commit reverts the `lint` and `format` scripts in `package.json`
to their original state, as the previous commit inadvertently
modified them beyond the scope of the original issue.

The scripts for Vitest remain in place.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants